-
Notifications
You must be signed in to change notification settings - Fork 215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates from ShaderMaterial to PointsMaterial in Particles.js #217
Conversation
I don't see my changes in the build files from your branch (the one being reviewed in #218), in fact, when I try them, I see it like this and should be like above: However, I see the Particles.js file did change, but those changes are not reflected in the build files either. I'm not fully familiar with JS dev pipelines, but my guess is that you didn't build (that is putting all function into a single file) the ROS3D.js library? It'd be easier for me if you build it and push it (you'd need to do it eventually anyway, right?), so I can just drop the library file into my setup for a quick test. |
I forgot to mention that I have introduced an API change which has now a proper deprecation warning. I looks like it impacted you because the Alternatively to this topic-wise coloring, the |
@carlosjoserg Hello, I just merged #218 that updates the major pipeline to visualize pointcloud2 and laserscan. Could you check if it fixes your issue? Note that Particles.js has been renamed to Points.js in favor of change in three.js. |
Hi @jihoonl Nope, that PR doesn't work for me. I understand Particle.js being renamed to Points.js is within LaserScan, and I'm using it like: lidar_front_ = new ROS3D.LaserScan({
ros : ros_,
topic : '/device/lidar_front_left/scan',
tfClient : tf_client_,
size : 0.1,
color : 0xFFA500,
rootObject : viewer3D_.scene}); With the underscored names being previously intialized. I'll try to figure out what changed then. I'll come back if I find it. In the meantime I'll close the PR, since it got outdated. Cheers, and thanks. |
Could you try this ? (the material defining options have been moved to their own object)
|
Thanks for the fast reply. The colors are recognized but not the number of points, I'm only seeing one point per topic. I don't get any messages on the web console either. The setup is the same as in the comment above. |
Thanks for the bug report, there was some spelling issue. |
Hi,
I was having issues with the current implementation using the latest of the THREE lib, and after struggling a bit with my very bad javascript skills, I managed to make it work.
I couldn't generate the build files, since I'm on Ubuntu 16.04 and the commands in the readme file didn't work for me. If someone tells me how to do it, I'd be glad to add the build files to this PR.
I can tell you it works for me, here you can see a screenshot with two laser scans of size 0.1 (different from the default 0.05) and separate colors (different from the default 0xFFA500).